Skip to content

minor phpstan fix#429

Merged
arhimede merged 1 commit into
7.0from
issue-428
May 22, 2026
Merged

minor phpstan fix#429
arhimede merged 1 commit into
7.0from
issue-428

Conversation

@Jurj-Bogdan
Copy link
Copy Markdown
Member

As the validateConfig method already checks the identity_class key is a valid class string, another assertion does not seem necessary - simply annotating it as such should fix the issue.

Closes #428

Signed-off-by: Jurj-Bogdan <bogdanjurj11@gmail.com>
@Jurj-Bogdan Jurj-Bogdan added the fixed applies to fixed bugs label May 22, 2026
@Jurj-Bogdan Jurj-Bogdan requested a review from arhimede May 22, 2026 11:16
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 22, 2026

Qodana for JS

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

@arhimede arhimede requested a review from alexmerlin May 22, 2026 11:17
@arhimede arhimede merged commit fecf72d into 7.0 May 22, 2026
34 checks passed
@alexmerlin alexmerlin deleted the issue-428 branch May 22, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fixed applies to fixed bugs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AuthenticationAdapter PHPStan error

3 participants